From 4d56b51100df05032ee78c5fa556b4e003de2761 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= Date: Thu, 4 Dec 2025 05:18:04 +0100 Subject: [PATCH] environment: Do not try to animate a St.Adjustment with unmapped actor We were already doing this for actual actors, but the case was skipped in the St.Adjustment case. However, as it is an animatable we can just use its related actor as reference, and keep the same logic despite their actual object type Part-of: --- js/ui/environment.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/js/ui/environment.js b/js/ui/environment.js index 369f9de49..094b95018 100644 --- a/js/ui/environment.js +++ b/js/ui/environment.js @@ -197,9 +197,12 @@ function _easeAnimatableProperty(animatable, propName, target, params) { // whether the transition should finish where it started const isReversed = autoReverse && numIterations % 2 === 0; + // The object is a Clutter.Animatable. + const actor = animatable.get_actor(); + // Copy Clutter's behavior for implicit animations, see // should_skip_implicit_transition() - if (animatable instanceof Clutter.Actor && !animatable.mapped) + if (!actor?.mapped) duration = 0; const prepare = () => { -- 2.39.5